Add Defaultable for CommandInputRecordField
#54
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This request is re-done of common-workflow-language/cwl-v1.1#20 to fix common-workflow-language/common-workflow-language#641.
I introduced
Defaultableand made it a parent record ofInputParameter,WorkflowStepInputandInputRecordField.I also add a conformance test for it.
I changed the description of
Defaultable#defaultby combining descriptions ofInputParameter#defaultandWorkflowStepInput#default.Please let me know if there are better descriptions to unifying other descriptions of the
defaultfield.Here is a list to be done (copied from common-workflow-language/cwl-v1.1#20):
cwltool --enable-devin av1.3.0-dev1documentcwltoolhow to copy the default value overcwltoolusingv1.0,v1.1andv1.2and a record field default (this should fail)